Skip to content

[Gtk4] Fix missing space between TreeItem icon and label#3409

Merged
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:treespace
Jun 30, 2026
Merged

[Gtk4] Fix missing space between TreeItem icon and label#3409
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:treespace

Conversation

@akurtakov

Copy link
Copy Markdown
Member

Measuring the allocation of GtkSeparator before it's realized always gives 0x0 so icon and label are really glued.
Use gtk_cell_renderer_get_padding to query the cell renderer for that info. Gtk 3 could use same code but I decided to not do it as separator has bigger width than cellrenderer padding and I don't want to change Gtk 3 visual behavior (make trees more "condensed").

@akurtakov

Copy link
Copy Markdown
Member Author

Before:
Screenshot From 2026-06-30 14-44-08
After:
Screenshot From 2026-06-30 14-43-39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Measuring the allocation of GtkSeparator before it's realized always
gives 0x0 so icon and label are really glued.
Use gtk_cell_renderer_get_padding to query the cell renderer for that
info. Gtk 3 could use same code but I decided to not do it as separator
has bigger width than cellrenderer padding and I don't want to change
Gtk 3 visual behavior (make trees more "condensed").
@github-actions

Copy link
Copy Markdown
Contributor

Test Results (linux)

  103 files  ±0    103 suites  ±0   14m 53s ⏱️ -12s
4 609 tests ±0  4 383 ✅ ±0  226 💤 ±0  0 ❌ ±0 
3 416 runs  ±0  3 338 ✅ ±0   78 💤 ±0  0 ❌ ±0 

Results for commit 26ba635. ± Comparison against base commit 0a73072.

@akurtakov akurtakov merged commit b31dbba into eclipse-platform:master Jun 30, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants